chore: add answers for test your understanding week 1 day 1-3 #93
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I'd rather not put a reference answer in the repo right now, but happy to review your answers :) |
skyzh
reviewed
Dec 29, 2024
| - What happens if the user adds a key larger than the target block size? | ||
| - We don't accept the write and send a `false` back to the user. | ||
|
|
||
| - Consider the case that the LSM engine is built on object store services (S3). How would you optimize/change the block format and parameters to make it suitable for such services? |
There was a problem hiding this comment.
One other optimization is to split data blocks and meta blocks into two files so that we can inspect bloom filter / key ranges without downloading the full file.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added answers for
Test your understanding week 1for days 1-3. Let me know if you see another folder better fit for this. There are a few questions I wasn't sure about so didn't answer it, particularly the Rust specific ones.This will need rework on the below perspectives: